Skip to content

Conversation

@AllanOXDi
Copy link
Member

@AllanOXDi AllanOXDi commented Nov 7, 2025

Summary

This PR fixes responsive issues at Importing from other channels (with search recommendations)
Closes #5325

References

#5325

Before

rd.2.mp4

After

Screen.Recording.2025-11-07.at.16.12.17.mov

Reviewer guidance

1§. Go to a channel and try to import resources from other channels

@AllanOXDi AllanOXDi marked this pull request as ready for review November 14, 2025 14:23
Copy link
Member

@akolson akolson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this should be an acceptable solution for now. With the channel and search results cards not yet transitioned to the KCard, they will be some glitches, but those should be resolved once the KDS migration is complete. Thanks @AllanOXDi

@akolson
Copy link
Member

akolson commented Nov 21, 2025

@AllanOXDi I think we should also target this to hotfixes. Thanks

@AllanOXDi AllanOXDi changed the base branch from unstable to hotfixes November 24, 2025 23:28
@AllanOXDi AllanOXDi changed the base branch from hotfixes to unstable November 24, 2025 23:28
@AllanOXDi AllanOXDi changed the base branch from unstable to hotfixes November 24, 2025 23:48
@AllanOXDi AllanOXDi force-pushed the fixes-responsive-issue-at-importing-channel branch from ced55ea to 4ae92df Compare November 24, 2025 23:49
@AllanOXDi AllanOXDi merged commit 8ee2083 into learningequality:hotfixes Nov 25, 2025
13 checks passed
# google-cloud-kms
# gunicorn
pillow==11.3.0
pillow==11.1.0
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@AllanOXDi Looks like this is a downgrade to whats in hotfixes. I think this should be 11.3.0?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure why this keeps happening

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

reverting this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Import from other channels (with search recommendations) - Responsive design issues

2 participants